Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented Potion Belt and associated features #25

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Implemented Potion Belt and associated features #25

wants to merge 11 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 28, 2014

Implemented a Potion Belt which extended the bag class.
Reordered Bag appearances in shops to Potion Belt(6), Seed Pouch(11), Scroll Holder(16), Wand Holster(21).
Adjusted tab size to fit 5 tabs as opposed to the original 4.
Implemented buff-item interaction where there is a chance that one of the hero's potions will shatter when the hero is frozen. Potion Belt will prevent potions from being shattered when hero is frozen.
Note: In order to get this buff-item interaction, we had to change the shatter method to public. If this violates your architecture, let us know the proper way of accessing the shatter method.

@Override
public String info() {
return
"This strong lether belt is capable of storing all your potions while protecting them from shattering due to frost";
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some typos here!

@rodriformiga
Copy link

Hi, @nathbenjwolf
Missed you implement the badges (ALL_BAGS_BOUGHT), validating that PotionBelt has already been purchased.
;)

@nathbenjwolf
Copy link

Hi @rodriformiga
Thanks for the feedback.
Good catch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants